Skip to content

fix: 修复沉浸式歌词歌曲名称上下被裁剪 - #459

Open
foragedysetour wants to merge 1 commit into
SPlayer-Dev:devfrom
foragedysetour:fix/immersive-title-clip
Open

foragedysetour wants to merge 1 commit into
SPlayer-Dev:devfrom
foragedysetour:fix/immersive-title-clip

Conversation

@foragedysetour

Copy link
Copy Markdown
Contributor

改动类型

  • 新功能(feat)
  • 缺陷修复(fix)
  • 重构 / 优化(不改变对外行为)
  • 文档(docs)
  • 其他(请在「改动说明」中注明)

是否包含破坏性变更

  • 是(请在「改动说明」中详细描述)
  • 否

改动说明

沉浸式歌词界面左侧的歌名,字的上下边缘会被削掉一点。

标题行高 leading-tight(1.25),字体 ascent + descent 通常比这高(微软雅黑约 1.31em),而 SMarquee 容器和 PlayerData 根节点都是 overflow-hidden,会导致行盒外元素被裁切。

改动:SMarquee.vue:61、layouts/standard/PlayerData.vue:153 的 overflow-hidden 换成 overflow-x-clip overflow-y-visible。

此次改动与 #439 不同之处是最小化修改,保留了滚动功能。

关联 Issue

Closes #438

测试情况

  • pnpm format、tsc --noEmit -p tsconfig.node.json、vue-tsc --noEmit -p tsconfig.web.json、eslint 均通过
  • 打 Windows 安装包实测:纯中文长标题、带括号和拉丁字母的标题,上下都不再缺字;短标题与改动前一致

截图 / 录屏

Before:
image

After:
image

自查清单

  • 本 PR 只包含一个主要功能 / 修复,没有夹带无关改动
  • 已在本地完整测试通过;AI 生成的代码同样自行测试并审阅过,未做未经验证的提交
  • 已运行 pnpm format,并确认 pnpm typecheck、pnpm lint 通过
  • 改动涉及原生模块时已 pnpm build:native 验证;未手写 native/*/index.d.ts
  • 已向 dev 分支提交

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] 歌词界面歌曲名称显示不全

1 participant